Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ci: carbonix_build zip build artifacts #282

Conversation

loki077
Copy link
Contributor

@loki077 loki077 commented Dec 30, 2024

This PR solves few small fixes and adds small features.

  • fix long job name
  • Added status check in Aircraft Config, if not active does not build an output.
  • 7zips the folder and upload to s3 reducing the size to 28MB from 168MB.

This will help to identify the status of the aircraft configuration file
whether it is in active or depericated state.
If the status is 'depericated' then the CI won't build the firmware
for that aircraft configuration.
This will only process file with status 'active'.
Previously in job name the complete path was being used, which was
causing issues to see the job name in the github actions.
This commit fixes the job name to only show the Aircraft name.
As the python script won't generate an output if the aircraft config
status is deprecated. Adding cecking for final-output directory before
AWS CLI installation and S3 upload to avoid unnecessary steps.
This filters the correct folders and 7zips them to be uploaded to S3.
@loki077 loki077 force-pushed the feature/SW-528-zip-aircraft-config-packages-and-upload-to-s-3-for-faster-download branch from 0450356 to 7ff704a Compare December 30, 2024 23:33
@loki077
Copy link
Contributor Author

loki077 commented Dec 30, 2024

Result in S3:
image

Structure in zip folder:
image
image

@loki077
Copy link
Contributor Author

loki077 commented Dec 30, 2024

Regular Folder size : 316MB
zip size: 140MB
7zip size: 28MB

@robertlong13 robertlong13 merged commit bcd2114 into CxPilot-7 Dec 31, 2024
57 checks passed
@robertlong13 robertlong13 deleted the feature/SW-528-zip-aircraft-config-packages-and-upload-to-s-3-for-faster-download branch December 31, 2024 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants